fix: changement des indicateurs nationaux pour les établissements tm-292 #3285
Conversation
| last_transmission_date: { | ||
| $gte: new Date(new Date().getFullYear(), 7, 1), |
There was a problem hiding this comment.
1/7 c'est bien le 1er aout ?
There was a problem hiding this comment.
on pourrait presque utiliser une constante car ya plusieurs endroits ds le code il me semble ou l'on utilise cette date pour la bascule des années.
There was a problem hiding this comment.
Oui le JS compte à partir de 0 pour les mois 🤦
Je suis désolé la PR est en speed je pense qu'il faudra l'améliorer ensuite : mettre une constante ou des vrais trucs de date
|
Voir s'il n'y a pas d'effet de bord sur le filtre > 1er aout 🤔 ? -> Bon apparemment non, la fonction n'est utilisée que dans getIndicateursNational qui elle mm n'est utilisée que dans la route /api/v1/indicateurs/national ✅ |
| @@ -451,7 +451,7 @@ function SectionApercuChiffresCles() { | |||
| {formatNumber(indicateursNational?.indicateursOrganismes?.total)} | |||
| </Text> | |||
| <Text fontSize="zeta" fontWeight="700" lineHeight="1em" color="bluefrance"> | |||
| organismes de formation en apprentissage | |||
| organismes de formation en apprentissage transmettent leurs données | |||
| @@ -468,6 +468,7 @@ function SectionApercuChiffresCles() { | |||
| (base des Carif-Oref) ; | |||
| </ListItem> | |||
| <ListItem>identifiés par un SIRET (ouvert) et un UAI valable ;</ListItem> | |||
| <ListItem>ayant envoyé des données depuis le premier août ;</ListItem> | |||
There was a problem hiding this comment.
le premier août ;
=>
le premier août.
🚀 Prévisualisation |
|
🎉 This PR is included in version 3.63.1 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
cf: https://tableaudebord-apprentissage.atlassian.net/browse/TM-292